Skip to content

🔧 chore(deps): refresh shared tooling patches#28

Merged
xdanger merged 1 commit into
mainfrom
codex/dependency-sweep-20260531-taptap.github
Jun 6, 2026
Merged

🔧 chore(deps): refresh shared tooling patches#28
xdanger merged 1 commit into
mainfrom
codex/dependency-sweep-20260531-taptap.github

Conversation

@xdanger
Copy link
Copy Markdown
Member

@xdanger xdanger commented May 31, 2026

Updates the TapTap shared workflow tooling with the smallest safe patch set still outstanding on this PR.

  • Bumps packageManager from pnpm@11.5.0 to the current published pnpm@11.5.2.
  • Bumps lint-staged from 17.0.6 to 17.0.7.
  • Bumps typescript-eslint from 8.60.0 to 8.60.1.
  • Keeps the existing eslint@10.4.1 patch from this PR.

Validation:

  • pnpm install --frozen-lockfile
  • pnpm outdated --format json -> {}
  • pnpm run lint

Copilot AI review requested due to automatic review settings May 31, 2026 07:35
@xdanger xdanger self-assigned this May 31, 2026
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 31, 2026

Greptile Summary

此 PR 将 lint 工具链的多个依赖升级至最新 patch 版本,并同步刷新 pnpm-lock.yaml。PR 标题仅提及 eslint,实际还包含 lint-stagedtypescript-eslintpackageManagerpnpm 本身的版本更新。

  • eslint^10.4.0 升级至 ^10.4.1typescript-eslint^8.60.0 升级至 ^8.60.1lint-staged^17.0.5 升级至 ^17.0.7pnpm packageManager 从 11.5.0 升级至 11.5.2
  • pnpm-lock.yaml 中所有相关依赖(@eslint/plugin-kittinyexec 等传递依赖)均已正确同步更新,完整性哈希值与新版本对应。

Confidence Score: 5/5

纯 patch 级别的 lint 工具链依赖升级,不涉及运行时代码,可安全合并。

所有变更均为 patch 版本升级(eslint、typescript-eslint、lint-staged、pnpm),lockfile 中的完整性哈希值与新版本一致,传递依赖同步正确,作者已验证 --frozen-lockfile 安装及 lint 通过。

所有文件均无需特别关注。

Important Files Changed

Filename Overview
package.json 将 eslint、lint-staged、typescript-eslint 升级至最新 patch 版本,同时将 packageManager 中的 pnpm 从 11.5.0 升级至 11.5.2
pnpm-lock.yaml 同步更新 lockfile,所有依赖版本及完整性哈希值与 package.json 中的新版本一致,未发现异常

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[pnpm outdated] -->|检测到新版本| B{依赖更新}
    B --> C[eslint 10.4.0 → 10.4.1]
    B --> D[typescript-eslint 8.60.0 → 8.60.1]
    B --> E[lint-staged 17.0.5 → 17.0.7]
    B --> F[pnpm 11.5.0 → 11.5.2]
    C --> G[pnpm install]
    D --> G
    E --> G
    F --> G
    G --> H[pnpm-lock.yaml 刷新]
    H --> I[pnpm run lint 验证通过]
Loading

Reviews (3): Last reviewed commit: "🔧 chore(deps): refresh tap config deps" | Re-trigger Greptile

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s lint tooling by bumping ESLint to the latest patch release and refreshing the pnpm lockfile to match, keeping the dev toolchain current with minimal risk.

Changes:

  • Bump eslint devDependency from ^10.4.0 to ^10.4.1.
  • Regenerate pnpm-lock.yaml to reflect the updated ESLint resolution (and related transitive updates).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updates the ESLint devDependency version range to ^10.4.1.
pnpm-lock.yaml Refreshes lock entries to resolve ESLint 10.4.1 (and related transitive dependency versions/integrity).
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

@xdanger
Copy link
Copy Markdown
Member Author

xdanger commented May 31, 2026

Dependency validation is clean (pnpm install --frozen-lockfile, pnpm run lint, pnpm outdated --format json -> {}). The remaining failing review check is external CI configuration: the Claude review action starts, but exits with Invalid API key · Fix external API key. No dependency patch change is indicated by the log.

@xdanger xdanger force-pushed the codex/dependency-sweep-20260531-taptap.github branch from 243a495 to 5438f7c Compare June 1, 2026 07:31
@github-actions github-actions Bot requested a review from Copilot June 1, 2026 07:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread package.json Outdated
@xdanger
Copy link
Copy Markdown
Member Author

xdanger commented Jun 1, 2026

Dependency diff is updated and local validation is clean (pnpm install --frozen-lockfile, pnpm run lint, pnpm outdated --format json => {}). The remaining failing review check is external CI configuration: Claude Code returned an error result: Invalid API key · Fix external API key. No buffered inline comments were produced.

- 🔧 keep shared workflow tooling current
@xdanger xdanger force-pushed the codex/dependency-sweep-20260531-taptap.github branch from 5438f7c to 2f9285f Compare June 6, 2026 07:28
@github-actions github-actions Bot requested a review from Copilot June 6, 2026 07:28
@xdanger xdanger changed the title 🔧 chore(deps): bump eslint patch 🔧 chore(deps): refresh shared tooling patches Jun 6, 2026
@claude
Copy link
Copy Markdown

claude Bot commented Jun 6, 2026

✅ Approved

Routine patch-level dependency bumps (eslint, lint-staged, typescript-eslint, pnpm). Lockfile is consistent with package.json changes. No code changes, no new findings.

🤖 Generated with Claude Code

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread package.json
"globals": "^17.6.0",
"husky": "^9.1.7",
"lint-staged": "^17.0.5",
"lint-staged": "^17.0.7",
@xdanger xdanger merged commit 7b67fea into main Jun 6, 2026
6 checks passed
@xdanger xdanger deleted the codex/dependency-sweep-20260531-taptap.github branch June 6, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants